Skip to content

DAOS-19084 test: Suppress Cgo syscall false positives#18448

Open
kjacque wants to merge 1 commit into
masterfrom
kjacque/even-more-cgo-suppressions
Open

DAOS-19084 test: Suppress Cgo syscall false positives#18448
kjacque wants to merge 1 commit into
masterfrom
kjacque/even-more-cgo-suppressions

Conversation

@kjacque
Copy link
Copy Markdown
Contributor

@kjacque kjacque commented Jun 5, 2026 •
edited by tanabarr
Loading

The syscall function signature changed, so suppressions needed to be updated. Also added a suppression related to an internal runtime context.Context implementation.

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • Appropriate Functional Test Stages were run.
  • At least two positive code reviews including at least one code owner from each category referenced in the PR.
  • Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

  • Gatekeeper requested (daos-gatekeeper added as a reviewer).

@kjacque
DAOS-19084 test: Suppress Cgo syscall false positives
1065503 
The syscall function signature changed, so suppressions needed
to be updated. Also added a suppression related to an internal
runtime context.Context implementation.

Signed-off-by: Kris Jacque <kris.jacque@hpe.com>
@kjacque kjacque self-assigned this Jun 5, 2026
@kjacque kjacque requested review from a team as code owners June 5, 2026 15:45
@kjacque kjacque requested review from a team, mjmac and tanabarr June 5, 2026 15:45
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 5, 2026

Ticket title is 'NLT Valgrind memcheck error: Syscall param write(buf) points to unaddressable byte(s)'
Status is 'In Review'
https://daosio.atlassian.net/browse/DAOS-19084

@daltonbohning daltonbohning mentioned this pull request Jun 5, 2026
Open
6 tasks
@tanabarr
Copy link
Copy Markdown
Contributor

tanabarr commented Jun 6, 2026

no go-related NLT failures, NLT failing on crt_iv_update_internal() ivo_on_update failed: DER_CONT_NONEXIST(-2050): 'The specified container does not exist' warning in strict mode
requesting forced landing

@tanabarr tanabarr requested a review from a team June 6, 2026 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tanabarr tanabarr tanabarr approved these changes

@soumagne soumagne soumagne approved these changes

@mjmac mjmac Awaiting requested review from mjmac

Assignees

@tanabarr tanabarr

@kjacque kjacque

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kjacque @tanabarr @soumagne